Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

4976 improve net8 metrics tests #2

Open
wants to merge 45 commits into
base: main
Choose a base branch
from
Open

4976 improve net8 metrics tests #2

wants to merge 45 commits into from

Conversation

ngruson
Copy link
Owner

@ngruson ngruson commented Nov 12, 2023

Fixes #
Design discussion issue #

Changes

Please provide a brief description of the changes here.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

ngruson and others added 30 commits November 10, 2023 16:02
@ngruson
UseHostBuilder to create client/server
56aa1db
@ngruson
Refactored RouteInformationIsNotAddedToRequestsOutsideOfMVC
1b69355
@utpilla
Merge branch 'main' into 4976-improve-net8-metrics-tests
4f161df
@ngruson
Refactord BasicTests
a60a8f5
@ngruson
Merge branch '4976-improve-net8-metrics-tests' of https://github.com/…
79660f6 
…ngruson/opentelemetry-dotnet into 4976-improve-net8-metrics-tests
@ngruson
Run tests on variable port instead of 5000
895da60
@ngruson
Merge branch 'main' of https://github.com/ngruson/opentelemetry-dotnet …
12aeef1 
…into 4976-improve-net8-metrics-tests
@ngruson
Removed unnecessary using
cdecae3
@ngruson
Added back blank line before EOF
3e7c3bf
@cijothomas
Fixed resource readme to reflect actual default (open-telemetry#5045)
0bc3bdc
@vishweshbankwar
[ASP.NET Core] Remove suppression for metrics generated from promethe…
78ab35c 
…us (open-telemetry#5044)
@CodeBlanch
[sdk] Ignore event counters in SelfDiagnosticsEventListener (open-tel…
ecb5dc8 
…emetry#5046)
@vishweshbankwar
[HttpClient and ASP.NET Core] Fix failures due to changes in .NET8.0 …
201811a 
…stable release (open-telemetry#5049)
@vishweshbankwar
Upgrade rc dependencies to net8.0 stable (open-telemetry#5051)
b0f9607
@utpilla
[repo] Misc changes (open-telemetry#5053)
c337d1e
@vishweshbankwar
[HttpClient & HttpWebRequest] Set network protocol version from respo…
e904994 
…nse object (open-telemetry#5043)
@Kielek
Fix histogram for dns.lookup.duration (open-telemetry#5058)
bdd931e
@utpilla
Make MetricPoint reclaim an opt-in experimental feature (open-telemet…
f2c2255 
…ry#5052)
@Yun-Ting
add concurrency tests with Coyote to CI (open-telemetry#4879)
d346196
@vishweshbankwar
Refactor HttpWebRequestActivitySource (open-telemetry#5055)
2a228f9
@alanwest
Fix Activity http.route and name (open-telemetry#5026)
68eabf0
@ngruson
Processed review comments
6024f19
@ngruson
Merge branch '4976-improve-net8-metrics-tests' of https://github.com/…
dd15e63 
…ngruson/opentelemetry-dotnet into 4976-improve-net8-metrics-tests
@vishweshbankwar
Http Span Status - Default to Unset if not in range defined by spec (o…
0256f87 
…pen-telemetry#5060)
@utpilla
Update CHANGELOG for 1.6.0-beta.3 release of instrumentation libraries (
276b307 
open-telemetry#5061)
@ngruson
Resolved merge conflicts
fabc1c6
@ngruson
Solved merge conflicts
cbb5320
@ngruson
Synced fork with main and merged conflicts
cf9bc01
@ngruson
Removed unnecessary usings
44639b4
@ngruson
Fixed ValidateNet8RateLimitingMetricsAsync
8952aca
ngruson and others added 15 commits November 18, 2023 12:20
@ngruson
Removed files that were not supposed to come along in this PR
f8c43fb
@vishweshbankwar
Merge branch 'main' into 4976-improve-net8-metrics-tests
7add987
@qhris
Add metric bucket override for OpenTelemetry.Instrumentation.AspNet (o…
2b62e47 
…pen-telemetry#5063)
@vishweshbankwar
Add note about .NET8.0 metrics in readme (open-telemetry#5067)
7dff324
@ngruson
Fixed DiagnosticSourceExceptionCallBackIsNotReceivedForExceptionsHand…
6227ab0 
…ledInMiddleware
@ngruson
Merge branch '4976-improve-net8-metrics-tests' of https://github.com/…
bac5a61 
…ngruson/opentelemetry-dotnet into 4976-improve-net8-metrics-tests
@vishweshbankwar
[HttpClient] Remove OTEL_SEMCONV_STABILITY_OPT_IN (open-telemetry#5068)
a55341a
@vishweshbankwar
[ASP.NET Core] Remove OTEL_SEMCONV_STABILITY_OPT_IN (open-telemetry#5066
0c4f065 
)
@CodeBlanch
[hosting] Support .NET 8 IMetricsBuilder API (open-telemetry#4958)
1f7f931
@ngruson
Removed skip of unit test DiagnosticSourceExceptionCallBackIsNotRecei…
6524198 
…vedForExceptionsHandledInMiddleware
@ngruson
Reverted sln changes
10c93cc
@ngruson
Resolved merge conflict in MetricTests.cs
ebe17a2
@ngruson
Conditional using directive
2ff66d5
@ngruson
Missing blank line before EOF
bbe10ec
@ngruson
Processed review comments
292da0f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@ngruson @utpilla @cijothomas @vishweshbankwar @CodeBlanch @Kielek @Yun-Ting @alanwest @qhris